👋 Welcome back serb! A progress list of the required criteria for merging this PR into master will be added to the body of your pull request. There are additional pull request commands available for use with this pull request.

@mrserb This change now passes all automated pre-integration checks.

ℹ️ This project also has non-automated pre-integration requirements. Please see the file CONTRIBUTING.md for details.

After integration, the commit message for the final commit will be:

8270083: -Wnonnull errors happen with GCC 11.1.1

Reviewed-by: shade

You can use pull request commands such as /summary, /contributor and /issue to adjust it as needed.

At the time when this comment was updated there had been 10 new commits pushed to the master branch:

• e78cfbe: 8369946: Bytecode rewriting causes Java heap corruption on PPC
• 74b7d69: 8314136: Test java/net/httpclient/CancelRequestTest.java failed: WARNING: tracker for HttpClientImpl(42) has outstanding operations
• 133dad6: 8333526: Restructure java/nio/channels/DatagramChannel/StressNativeSignal.java to a fail fast exception handling policy
• ... and 7 more: https://git.openjdk.org/jdk17u-dev/compare/bd1728a9454126723568b23d8a171498103c06e0...master

As there are no conflicts, your changes will automatically be rebased on top of these commits when integrating. If you prefer to avoid this automatic rebasing, please check the documentation for the /integrate command for further details.

➡️ To integrate this PR with the above commit message to the master branch, type /integrate in a new comment.

This backport pull request has now been updated with issue from the original commit.

This patch, along with several similar ones, was removed from the mainline as part of JDK-8291895. That issue also references other related tickets titled “Undefined Behaviour in class XX,” which as of now remain unfixed in 17u. Therefore, I think this patch should be merged first, as it will also simplify future backports related to Xcode 16+.

@shipilev @theRealAph please take a look

⚠️ @mrserb This change is now ready for you to apply for maintainer approval. This can be done directly in each associated issue or by using the /approval command.

Webrevs

• 00: Full (09ed1b90)

/approval request
Clean backport. The patch updates the code to handle certain warnings during compilation. It is part of the ongoing effort to migrate JDK 17 to the new toolchain, including Xcode 16, and will make further patches cleaner.

@mrserb
8270083: The approval request has been created successfully.

/integrate

Going to push as commit 5d35f53.
Since your change was applied there have been 13 commits pushed to the master branch:

• f796f9e: 8318467: [jmh] tests concurrent.Queues and concurrent.ProducerConsumer hang with 101+ threads
• 2d94694: 8318613: ChoiceFormat patterns are not well tested
• 25b2e4b: 8313355: javax/management/remote/mandatory/notif/ListenerScaleTest.java failed with "Exception: Failed: ratio=792.2791601423487"
• ... and 10 more: https://git.openjdk.org/jdk17u-dev/compare/bd1728a9454126723568b23d8a171498103c06e0...master

Your commit was automatically rebased without conflicts.

@mrserb Pushed as commit 5d35f53.

💡 You may see a message that your pull request was closed with unmerged commits. This can be safely ignored.